Skip to content

Add D41x support - #266

Merged
ejgoldik merged 3 commits into
devfrom
D41x_support
Aug 27, 2025
Merged

Add D41x support#266
ejgoldik merged 3 commits into
devfrom
D41x_support

Conversation

@ejgoldik

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings August 11, 2025 08:40

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for D41x Intel RealSense cameras to the existing D4xx kernel driver. The changes include adding device type definitions, resolution configurations, and format specifications specific to the D41x variant.

  • Adds D41x device type constant and configuration structures for depth and calibration modes
  • Updates the driver probe message to be more generic for all D4xx devices
  • Increments module version to reflect the new D41x support

Comment thread kernel/realsense/d4xx.c
.width = 256,
.height = 144,
.framerates = ds5_framerate_90,
.n_framerates = ARRAY_SIZE(ds5_framerate_90),

Copilot AI Aug 11, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inconsistent framerate array usage. All other entries in d41x_depth_sizes use ds5_framerate_to_90 but this entry uses ds5_framerate_90. This should likely be ds5_framerate_to_90 for consistency, unless there's a specific reason for this 256x144 resolution to only support 90fps.

Suggested change
.n_framerates = ARRAY_SIZE(ds5_framerate_90),
.framerates = ds5_framerate_to_90,
.n_framerates = ARRAY_SIZE(ds5_framerate_to_90),

Copilot uses AI. Check for mistakes.
@ejgoldik
ejgoldik merged commit 35c4ab3 into dev Aug 27, 2025
5 checks passed
@ejgoldik
ejgoldik deleted the D41x_support branch August 27, 2025 06:06
ymodlin pushed a commit that referenced this pull request Sep 18, 2025
* Add D41x support

* Bump module version

* Fix D41x resolutions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants